Skip to content

Add parameter for the last number of queries to the DB to the logs#40323

Merged
potiuk merged 1 commit into
apache:mainfrom
VladaZakharova:last-num-of-db-queries
Jul 2, 2024
Merged

Add parameter for the last number of queries to the DB to the logs#40323
potiuk merged 1 commit into
apache:mainfrom
VladaZakharova:last-num-of-db-queries

Conversation

@MaksYermak

Copy link
Copy Markdown
Contributor

In this PR I have added a new parameter for the last number of queries to the DB to the DAG file processing stats logs. Link to the feature issue: #40282


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@boring-cyborg boring-cyborg Bot added the area:Scheduler including HA (high availability) scheduler label Jun 19, 2024
Comment thread airflow/dag_processing/manager.py Outdated
Comment thread airflow/dag_processing/manager.py Outdated
Comment thread airflow/dag_processing/processor.py
Comment thread tests/dag_processing/test_job_runner.py
@MaksYermak MaksYermak force-pushed the last-num-of-db-queries branch from f99416a to b306f36 Compare June 20, 2024 13:53
@kosteev

kosteev commented Jun 21, 2024

Copy link
Copy Markdown
Contributor

LGTM

Comment thread tests/dag_processing/test_processor.py Outdated
@MaksYermak MaksYermak force-pushed the last-num-of-db-queries branch from b306f36 to ee27768 Compare June 24, 2024 12:26
@MaksYermak MaksYermak force-pushed the last-num-of-db-queries branch from ee27768 to 56c0753 Compare June 25, 2024 11:25
@kosteev

kosteev commented Jun 25, 2024

Copy link
Copy Markdown
Contributor

LGTM

@VladaZakharova

Copy link
Copy Markdown
Contributor

Hi @potiuk !
Can you please check changes here once again? Thank you!

@MaksYermak MaksYermak requested a review from potiuk July 1, 2024 08:38
@potiuk potiuk merged commit 7fde239 into apache:main Jul 2, 2024
@potiuk

potiuk commented Jul 2, 2024

Copy link
Copy Markdown
Member

Nice one :)

@utkarsharma2 utkarsharma2 added this to the Airflow 2.10.0 milestone Jul 12, 2024
@utkarsharma2 utkarsharma2 added type:misc/internal Changelog: Misc changes that should appear in change log type:new-feature Changelog: New Features and removed type:misc/internal Changelog: Misc changes that should appear in change log labels Jul 12, 2024
romsharon98 pushed a commit to romsharon98/airflow that referenced this pull request Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Scheduler including HA (high availability) scheduler type:new-feature Changelog: New Features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants